Skip to content

Haiku fix ZTS build, disabling tls model#8047

Closed
devnexen wants to merge 1 commit into
php:PHP-8.0from
devnexen:haiku_zts_build_fix
Closed

Haiku fix ZTS build, disabling tls model#8047
devnexen wants to merge 1 commit into
php:PHP-8.0from
devnexen:haiku_zts_build_fix

Conversation

@devnexen

@devnexen devnexen commented Feb 6, 2022

Copy link
Copy Markdown
Member

No description provided.

@cmb69 cmb69 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! I don't know about Haiku, but you certainly do, so this looks good to me. :)

@cmb69 cmb69 closed this in bea542a Feb 7, 2022
@mvorisek

mvorisek commented Mar 7, 2022

Copy link
Copy Markdown
Contributor

@devnexen can you see opcache ext as loaded /w JIT enabled with this, to me incomplete, fix on Haiku?

see related issue #8180 (comment) discussion

@devnexen

devnexen commented Mar 7, 2022

Copy link
Copy Markdown
Member Author

afaik it is irrelevant in case of haiku as opcache does not work at all in the first place in this platform.

@mvorisek

mvorisek commented Mar 7, 2022

Copy link
Copy Markdown
Contributor

afaik it is irrelevant in case of haiku as opcache does not work at all in the first place in this platform.

thanks for the reply, still, shouln't Haiku be checked here too

#elif defined(__x86_64__) && defined(__GNUC__) && !defined(__FreeBSD__) && !defined(__OpenBSD__) && !defined(__MUSL__)
?

@devnexen

devnexen commented Mar 7, 2022

Copy link
Copy Markdown
Member Author

sure why not, haiku is still in beta and does not support yet tls model.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants